Skip to content

Utilise X-FORWARDED-[FOR|PORT] headers -- Means things work properly when behind a reverse proxy.#397

Closed
DanBUK wants to merge 2 commits intosocketio:masterfrom
DanBUK:master
Closed

Utilise X-FORWARDED-[FOR|PORT] headers -- Means things work properly when behind a reverse proxy.#397
DanBUK wants to merge 2 commits intosocketio:masterfrom
DanBUK:master

Conversation

@DanBUK
Copy link
Copy Markdown

@DanBUK DanBUK commented Jul 19, 2011

Quite a simple patch really; but necessary when working behind reverse proxies.

@tedeh
Copy link
Copy Markdown

tedeh commented Jul 20, 2011

+1, would save me a lot of headache

@DanBUK
Copy link
Copy Markdown
Author

DanBUK commented Jul 20, 2011

I just fixed this pull request. I was referencing the wrong object before.

@rauchg
Copy link
Copy Markdown
Contributor

rauchg commented Jul 31, 2011

x-forwarded-for can be a comma separated list of values, depending on the hops the client made:

http://en.wikipedia.org/wiki/X-Forwarded-For#Format

Please fix your patch and add tests =]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants